Skip to content

sdl3_mixer 3.2.0 (new formula)#276253

Merged
BrewTestBot merged 2 commits intoHomebrew:mainfrom
atticus5:atticus5-patch-sdl3-mixer
Apr 24, 2026
Merged

sdl3_mixer 3.2.0 (new formula)#276253
BrewTestBot merged 2 commits intoHomebrew:mainfrom
atticus5:atticus5-patch-sdl3-mixer

Conversation

@atticus5
Copy link
Copy Markdown
Contributor

@atticus5 atticus5 commented Apr 6, 2026


  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>?
  • Is your test running fine brew test <formula>?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

  • AI was used to generate or assist with generating this PR. Please specify below how you used AI to help you, and what steps you have taken to manually verify the changes.

Generated the first template from sdl2_mixer, then did some cleanup, ran all the tests, including a test in my project with the installed library and was fine.


@github-actions github-actions Bot added autosquash Automatically squash pull request commits according to Homebrew style. new formula PR adds a new formula to Homebrew/homebrew-core labels Apr 6, 2026
@daeho-ro
Copy link
Copy Markdown
Member

daeho-ro commented Apr 6, 2026

Please keep a PR with force push,

Comment thread Formula/s/sdl3_mixer.rb Outdated
@cho-m cho-m changed the title Add sdl3_mixer 3.2.0 sdl3_mixer 3.2.0 Apr 6, 2026
@cho-m cho-m changed the title sdl3_mixer 3.2.0 sdl3_mixer 3.2.0 (new formula) Apr 6, 2026
@daeho-ro daeho-ro force-pushed the atticus5-patch-sdl3-mixer branch from 84b6ffd to bba24a9 Compare April 9, 2026 12:17
@github-actions github-actions Bot removed the autosquash Automatically squash pull request commits according to Homebrew style. label Apr 9, 2026
@atticus5
Copy link
Copy Markdown
Contributor Author

atticus5 commented Apr 9, 2026

Can this be merged?

@daeho-ro daeho-ro force-pushed the atticus5-patch-sdl3-mixer branch from bba24a9 to 816c3ea Compare April 12, 2026 13:23
@daeho-ro daeho-ro added the ready to merge PR can be merged once CI is green label Apr 12, 2026
Comment thread Formula/s/sdl3_mixer.rb Outdated
@daeho-ro daeho-ro force-pushed the atticus5-patch-sdl3-mixer branch from 816c3ea to b499362 Compare April 12, 2026 14:07
@daeho-ro daeho-ro removed the ready to merge PR can be merged once CI is green label Apr 12, 2026
@neogeek
Copy link
Copy Markdown

neogeek commented Apr 14, 2026

I'm fairly new to homebrew formula dev (previously just custom tap setup), so apologize if I'm off base here.

I was looking at testing out this PR locally and was able to get it to work by referencing the already existing sdl3_image formula and adding the following flags to the args array (I confirmed they exist via the sdl_mixer repo cmake file):

-DSDLMIXER_EXAMPLES=OFF
-DSDLMIXER_TESTS=OFF

https://github.com/Homebrew/homebrew-core/blob/main/Formula/s/sdl3_image.rb#L44-L45

Running the following commands:

brew reinstall --build-from-source ./sdl3_mixer.rb
brew linkage sdl3_mixer

Resulted in:

System libraries:
  /usr/lib/libSystem.B.dylib
Homebrew libraries:
  /opt/homebrew/opt/sdl3/lib/libSDL3.0.dylib (sdl3)
Dependencies with no linkage:
  game-music-emu
  libvorbis
  libxmp
  opusfile

And when I tested this build with my project (previously tested with the latest SDL_mixer via git and cmake), everything worked as expected.

Regarding the comment above about the SDLMIXER_DEPS_SHARED flag, it looks like the same flag is turned off in the sdl3_image formula (again used for reference), turned off in the cmake file of both the sdl_image and sdl_mixer repo as well as turned on in the build for both repo's release scripts.

@github-actions github-actions Bot added the autosquash Automatically squash pull request commits according to Homebrew style. label Apr 20, 2026
@atticus5
Copy link
Copy Markdown
Contributor Author

Works for me as well:

System libraries:
  /usr/lib/libSystem.B.dylib
Homebrew libraries:
  /opt/homebrew/opt/sdl3/lib/libSDL3.0.dylib (sdl3)
Dependencies with no linkage:
  fluid-synth
  game-music-emu
  libvorbis
  libxmp
  opusfile

I think it can be merged now

@atticus5
Copy link
Copy Markdown
Contributor Author

@daeho-ro

@daeho-ro
Copy link
Copy Markdown
Member

@atticus5 Can you squash your commit to the first one?

@atticus5 atticus5 force-pushed the atticus5-patch-sdl3-mixer branch from 7d05aba to f35188f Compare April 22, 2026 15:40
@github-actions github-actions Bot removed the autosquash Automatically squash pull request commits according to Homebrew style. label Apr 22, 2026
@atticus5
Copy link
Copy Markdown
Contributor Author

@daeho-ro done!

@atticus5 atticus5 closed this Apr 23, 2026
@atticus5 atticus5 reopened this Apr 23, 2026
@atticus5
Copy link
Copy Markdown
Contributor Author

When is this going to master branch?

@daeho-ro daeho-ro force-pushed the atticus5-patch-sdl3-mixer branch from f35188f to b14cedd Compare April 24, 2026 07:50
@daeho-ro
Copy link
Copy Markdown
Member

CI looks messed, rebase to origin/main.

@daeho-ro daeho-ro force-pushed the atticus5-patch-sdl3-mixer branch from b14cedd to aaa1b6e Compare April 24, 2026 08:07
@github-actions github-actions Bot added the automerge-skip `brew pr-automerge` will skip this pull request label Apr 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🤖 An automated task has requested bottles to be published to this PR.

Caution

Please do not push to this PR branch before the bottle commits have been pushed, as this results in a state that is difficult to recover from. If you need to resolve a merge conflict, please use a merge commit. Do not force-push to this PR branch.

@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Bottle publish failed.

Co-authored-by: Carlo Cabrera <github@carlo.cab>

fix linkage issues
@daeho-ro daeho-ro force-pushed the atticus5-patch-sdl3-mixer branch from aaa1b6e to 4cda0fd Compare April 24, 2026 08:22
@github-actions github-actions Bot removed the automerge-skip `brew pr-automerge` will skip this pull request label Apr 24, 2026
@daeho-ro daeho-ro added the ready to merge PR can be merged once CI is green label Apr 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🤖 An automated task has requested bottles to be published to this PR.

Caution

Please do not push to this PR branch before the bottle commits have been pushed, as this results in a state that is difficult to recover from. If you need to resolve a merge conflict, please use a merge commit. Do not force-push to this PR branch.

@github-actions github-actions Bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Apr 24, 2026
@BrewTestBot BrewTestBot enabled auto-merge April 24, 2026 13:20
@BrewTestBot BrewTestBot added this pull request to the merge queue Apr 24, 2026
Merged via the queue into Homebrew:main with commit f83d4b5 Apr 24, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. new formula PR adds a new formula to Homebrew/homebrew-core ready to merge PR can be merged once CI is green

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants